Conversation
|
👋 FelixFan1992, thanks for creating this pull request! To help reviewers, please consider creating future PRs as drafts first. This allows you to self-review and make any final changes before notifying the team. Once you're ready, you can mark it as "Ready for review" to request feedback. Thanks! |
There was a problem hiding this comment.
Pull request overview
Updates the default Docker image used by the Sui blockchain component, changing both the Sui tools version and the release channel used when Input.Image is not explicitly provided.
Changes:
- Bump the default
mysten/sui-toolsimage fromdevnet-v1.61.0tomainnet-v1.68.1for Sui node startup.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| if in.Image == "" { | ||
| in.Image = "mysten/sui-tools:devnet-v1.61.0" | ||
| in.Image = "mysten/sui-tools:mainnet-v1.68.1" | ||
| } |
There was a problem hiding this comment.
The default Sui image tag changed from devnet-... to mainnet-... while the code still starts a local node with --force-regenesis --with-faucet. If the intent is only a version bump, consider keeping the same release channel (e.g., devnet-v1.68.1) or otherwise document/justify the behavior change, since this alters the default runtime behavior for anyone not explicitly setting Input.Image.
No description provided.